Skip to content

Fix security vulnerabilities from audit#18

Merged
dadachi merged 1 commit intomainfrom
fix_security_vulnerabilities
Mar 27, 2026
Merged

Fix security vulnerabilities from audit#18
dadachi merged 1 commit intomainfrom
fix_security_vulnerabilities

Conversation

@dadachi
Copy link
Copy Markdown
Contributor

@dadachi dadachi commented Mar 27, 2026

Summary

  • Disable cleartext traffic and add network_security_config.xml (debug override allows local dev)
  • Disable app backup to prevent auth token extraction via adb backup
  • Encrypt DataStore tokens at rest using Tink AEAD with Android Keystore
  • Add certificate pinning for api.nativeapptemplate.com (leaf + intermediate CA)
  • Validate NDEF URL origin before extracting intent data
  • Restrict <profileable android:shell="true"> to debug builds only
  • Remove unused mailto intent filter from MainActivity

Test plan

  • Build passes (assembleDebug)
  • All unit tests pass
  • Verify NFC tag scanning works on device
  • Verify local dev server connection works in debug build
  • Verify production API connectivity with certificate pinning

🤖 Generated with Claude Code

- Disable cleartext traffic and add network_security_config.xml (debug override allows local dev)
- Disable app backup to prevent auth token extraction via adb backup
- Encrypt DataStore tokens at rest using Tink AEAD with Android Keystore
- Add certificate pinning for api.nativeapptemplate.com (leaf + intermediate CA)
- Validate NDEF URL origin before extracting intent data
- Restrict <profileable android:shell="true"> to debug builds only
- Remove unused mailto intent filter from MainActivity

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dadachi dadachi merged commit d76d6d3 into main Mar 27, 2026
1 check passed
@dadachi dadachi deleted the fix_security_vulnerabilities branch March 27, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant